Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: expired token causes scannerd to be unable to access SQLE normally #2620

Merged
merged 4 commits into from
Sep 20, 2024
Merged

fix: expired token causes scannerd to be unable to access SQLE normally #2620

merged 4 commits into from
Sep 20, 2024

Conversation

winfredLIN
Copy link
Collaborator

@winfredLIN winfredLIN commented Sep 19, 2024 •
edited
Loading

关联的 issue

#2602

描述你的变更

  1. 对于慢日志扫描，将慢日志扫描的接口组移动到v1路由组外，该接口的权限校验全由ScannerVerifier来鉴权
  2. 在ScannerVerifier中，针对token过期的错误，放行包含该错误的token

确认项（pr提交后操作）

Tip

请在指定复审人之前，确认并完成以下事项，完成后✅

  • 我已完成自测
  • 我已在关联的issue里补充了实现方案
  • 我已在关联的issue里补充了测试影响面
  • 我已确认了变更的兼容性，如果不兼容则在issue里标记 not_compatible
  • 我已确认了是否要更新文档，如果要更新则在issue里标记 need_update_doc

@winfredLIN winfredLIN self-assigned this Sep 19, 2024
@ColdWaterLW ColdWaterLW merged commit e953fc1 into release-2.9999.x Sep 20, 2024
3 checks passed
@ColdWaterLW ColdWaterLW deleted the issue-2602-v2 branch September 20, 2024 06:45
@winfredLIN winfredLIN changed the title fix: expired token causes scannerd to be unable to access SQLite normally fix: expired token causes scannerd to be unable to access SQLE normally Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ColdWaterLW ColdWaterLW ColdWaterLW approved these changes

Assignees

@winfredLIN winfredLIN

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@winfredLIN @ColdWaterLW